Fix how we build frameworks. Fixes #53813. (#1952) - #1985
Merged
Conversation
Previously we copied any equivalent .dylib and ran install_name_tool on the library to change the library id to make it a framework. Unfortunately this does not work when the library contains bitcode, because bitcode embeds linker flags (-install_name for instance), and install_name_tool does not change those linker flags. This means that we need to create frameworks by linking with the proper arguments, since it's much more difficult to fixup the embedded bitcode linker flags as well. So change how be build Mono.framework, Xamarin.framework, and any frameworks built from assemblies to: * Always link instead of fixup a dylib. For Mono.framework this means extracting all the object files from libmonosgen-2.0.a and linking those, for Xamarin.framework this means linking the object files we've already built. * Make sure the library is correctly named when linked (once again: bitcode contains embedded linker flags, so renaming the executable later breaks stuff as well). I've also extracted the logic that creates Mono.framework from libmonosgen-2.0.a to a separate shell script, to deduplicate this logic. This required a minor change in the mono builds: we need the Mono.framework when building the `all` target, so make sure that happens. https://bugzilla.xamarin.com/show_bug.cgi?id=53813
Member
Author
|
This is a backport of PR #1985. |
Contributor
|
Build success |
spouliot
approved these changes
Apr 11, 2017
VincentDondain
added a commit
to VincentDondain/xamarin-macios
that referenced
this pull request
Sep 18, 2019
New commits in xamarin/maccore: * xamarin/maccore@486ad67875 [device-builds] Fix if condition typo (double `[`) (dotnet#1985) * xamarin/maccore@fda32707a7 [tests] Update (and hopefully simplify) submission tests documentation wrt cerficates/profiles expiration (dotnet#1972) * xamarin/maccore@f23cd3a194 Merge pull request dotnet#1977 from xamarin/swift-o-matic-protocol-list-enum-factory * xamarin/maccore@aaa2bec8da Enum factory test * xamarin/maccore@d3c7890df2 Merge pull request dotnet#1974 from xamarin/swift-o-matic-protocol-list-enum * xamarin/maccore@436a376ab4 [provisioning-profiles] Add 'Developer ID Application' and 'Developer ID Installer' certificates. (dotnet#1975) * xamarin/maccore@ddc68fc899 Support for protocol list types in enum payloads. Diff: https://github.com/xamarin/maccore/compare/4611683312fd8ad4eb2b2c093e43a5f71405bdee..486ad67875db90261f3c9818380b979db3094946
VincentDondain
added a commit
that referenced
this pull request
Sep 19, 2019
New commits in xamarin/maccore: * xamarin/maccore@486ad67875 [device-builds] Fix if condition typo (double `[`) (#1985) * xamarin/maccore@fda32707a7 [tests] Update (and hopefully simplify) submission tests documentation wrt cerficates/profiles expiration (#1972) * xamarin/maccore@f23cd3a194 Merge pull request #1977 from xamarin/swift-o-matic-protocol-list-enum-factory * xamarin/maccore@aaa2bec8da Enum factory test * xamarin/maccore@d3c7890df2 Merge pull request #1974 from xamarin/swift-o-matic-protocol-list-enum * xamarin/maccore@436a376ab4 [provisioning-profiles] Add 'Developer ID Application' and 'Developer ID Installer' certificates. (#1975) * xamarin/maccore@ddc68fc899 Support for protocol list types in enum payloads. Diff: https://github.com/xamarin/maccore/compare/4611683312fd8ad4eb2b2c093e43a5f71405bdee..486ad67875db90261f3c9818380b979db3094946
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Previously we copied any equivalent .dylib and ran install_name_tool on the
library to change the library id to make it a framework.
Unfortunately this does not work when the library contains bitcode, because
bitcode embeds linker flags (-install_name for instance), and
install_name_tool does not change those linker flags.
This means that we need to create frameworks by linking with the proper
arguments, since it's much more difficult to fixup the embedded bitcode linker
flags as well.
So change how be build Mono.framework, Xamarin.framework, and any frameworks
built from assemblies to:
Always link instead of fixup a dylib. For Mono.framework this means
extracting all the object files from libmonosgen-2.0.a and linking those,
for Xamarin.framework this means linking the object files we've already
built.
Make sure the library is correctly named when linked (once again: bitcode
contains embedded linker flags, so renaming the executable later breaks
stuff as well).
I've also extracted the logic that creates Mono.framework from
libmonosgen-2.0.a to a separate shell script, to deduplicate this logic.
This required a minor change in the mono builds: we need the Mono.framework
when building the
alltarget, so make sure that happens.https://bugzilla.xamarin.com/show_bug.cgi?id=53813